[core] Logging : fix several issues on the log files #2946
+21
−26
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the problem
We had several issues on the logs :
raise ValueError(...)for example in theprocessChunkthen on the log we only had the error and not the full loglogfiles (in addition to0.log) in parallelized nodes with 1 chunkWhy we had the issues and how this PR addresses them
executeChunkCommandLinefixes itchunk.logManager.startlog. Even on cases where we have a single chunk I think it's simply better to use0.logthat way we have a better naming consistency anyways.